Skip to content

[Mellanox] Integrate HW-MGMT 7.0040.4001 Changes#479

Closed
DavidZagury wants to merge 1 commit intosonic-net:masterfrom
DavidZagury:hw_mgmt_4001
Closed

[Mellanox] Integrate HW-MGMT 7.0040.4001 Changes#479
DavidZagury wants to merge 1 commit intosonic-net:masterfrom
DavidZagury:hw_mgmt_4001

Conversation

@DavidZagury
Copy link
Contributor

Why I did it
Integrate HW-MGMT 7.0040.4001 Changes

Patch List

  • 0030-1-platform-mellanox-nvsw-sn2201-Add-check-for-platform.patch : torvalds/linux@d56fbfbaf592
  • 0098-platform-mellanox-Downstream-Add-support-for-new-Nvi.patch :
  • 0099-platform-mellanox-Downstream-Add-support-DGX-flavor-.patch :
  • 0100-platform-mellanox-mlxreg-io-Extend-number-of-hwmon-a.patch :
  • 0101-platform_data-mlxreg-Add-fields-for-interrupt-storm-.patch :
  • 0102-platform-mellanox-mlxreg-hotplug-Add-support-for-han.patch :
  • 0103-platform-mellanox-mlx-dpu-improve-interrupt-handling.patch :
  • 8012-hwmon-pmbus-Downstream-Workaround-for-psu-attributes.patch :

@DavidZagury DavidZagury requested a review from a team as a code owner May 4, 2025 16:33
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented May 4, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

 ## Patch List
* 0030-1-platform-mellanox-nvsw-sn2201-Add-check-for-platform.patch : torvalds/linux@d56fbfbaf592
* 0098-platform-mellanox-Downstream-Add-support-for-new-Nvi.patch :
* 0099-platform-mellanox-Downstream-Add-support-DGX-flavor-.patch :
* 0100-platform-mellanox-mlxreg-io-Extend-number-of-hwmon-a.patch :
* 0101-platform_data-mlxreg-Add-fields-for-interrupt-storm-.patch :
* 0102-platform-mellanox-mlxreg-hotplug-Add-support-for-han.patch :
* 0103-platform-mellanox-mlx-dpu-improve-interrupt-handling.patch :
* 8012-hwmon-pmbus-Downstream-Workaround-for-psu-attributes.patch :
@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: Oleksandr Shamray <[email protected]>
Reviewed-by: Vadim Pasternak <[email protected]>

Signed-off-by: Oleksandr Shamray <[email protected]>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oleksandr’s Signed-off-by line is doubled.


+static int __init mlxplat_dmi_ng800_dgx_matched(const struct dmi_system_id *dmi)
+{
+ int i;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could be size_t.

Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This system is based on Nvidia SN5600 Spectrum™-4 Based 64x800Gb/s ETH Switch System, with the
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use 75 characters per line.

From: Oleksandr Shamray <[email protected]>
Date: Mon, 9 Dec 2024 13:38:05 +0200
Subject: [PATCH 08/15] platform: mellanox: Downstream: Add support DGX flavor
of next-generation 800GB/s ethernet switch.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No dot/period at the end of commit message summaries/titles.

From fc7c0147a6816741ac6a3323b80e0debbbb6dbd4 Mon Sep 17 00:00:00 2001
From: Oleksandr Shamray <[email protected]>
Date: Mon, 9 Dec 2024 13:38:05 +0200
Subject: [PATCH 08/15] platform: mellanox: Downstream: Add support DGX flavor
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does the prefix Downstream mean?

From edf1951c7f5e87a9b6c2ad82bd97866012f552a6 Mon Sep 17 00:00:00 2001
From: Ciju Rajan K <[email protected]>
Date: Tue, 25 Mar 2025 11:20:04 +0200
Subject: [PATCH 01/22] From 4b17e10a22771fcd2dbb4754c43200af2fc69425 Mon Sep
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks wrong.


This patch introduces the driver caching.

Bugs# 3874682
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In which bugtracker?

Date: Thu, 10 Oct 2024 14:13:28 +0300
Subject: hwmon: pmbus: Downstream: Workaround for psu attributes

Starting from kernel 5.12.rc-1, pmbus reads are always
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please re-flow for 75 characters per line.

@DavidZagury DavidZagury marked this pull request as ready for review May 8, 2025 14:12
@saiarcot895
Copy link
Contributor

FYI, this update is present in #454.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants